Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat: Add Lisk and Lisk Sepolia logos #28763

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ricott1
Copy link
Contributor

@ricott1 ricott1 commented Nov 27, 2024

Description

  • Add Lisk mainnet and testnet name and icon support.

Open in GitHub Codespaces

Related issues

Fixes:

N/A

Manual testing steps

Lisk

  1. Add Lisk network, from instance from https://blockscout.lisk.com/
  2. Click on the network selection button on the top left.
  3. Check the logo for the newly added Lisk network.

Lisk Sepolia

  1. Add Lisk Sepolia, from instance from https://sepolia-blockscout.lisk.com/
  2. Click on the network selection button on the top left.
  3. Check the logo for the newly added Lisk Sepolia network.

Screenshots/Recordings

Before

Screenshot 2024-11-27 at 10 53 54

After

Screenshot 2024-11-27 at 10 51 56

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@ricott1 ricott1 requested a review from a team as a code owner November 27, 2024 10:01
@ricott1 ricott1 marked this pull request as draft November 27, 2024 10:01
Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@ricott1 ricott1 force-pushed the feat/add-lisk-logo branch 5 times, most recently from 2a49d72 to 056cea2 Compare November 28, 2024 15:13
@ricott1 ricott1 marked this pull request as ready for review November 28, 2024 15:41
@ricott1
Copy link
Contributor Author

ricott1 commented Nov 28, 2024

The optional 'test-e2e-swap-playwright' job keeps failing but I think it's completely unrelated to this PR.

@ricott1 ricott1 force-pushed the feat/add-lisk-logo branch 3 times, most recently from 3eb947c to 36f4aa9 Compare December 4, 2024 09:29
@ricott1 ricott1 requested a review from a team as a code owner December 4, 2024 09:29
@ricott1 ricott1 force-pushed the feat/add-lisk-logo branch from 36f4aa9 to e5be4c0 Compare December 5, 2024 07:02
@ricott1
Copy link
Contributor Author

ricott1 commented Dec 11, 2024

Hello, how can I get a second review for this? @hjetpoluru

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants